Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(dialog): add 'custom-content' slot #11072

Merged
merged 9 commits into from
Jan 28, 2025

Conversation

DitwanP
Copy link
Contributor

@DitwanP DitwanP commented Dec 16, 2024

Related Issue: #10323

Summary

Deprecates the "content" slot in favor of a new "custom-content" slot that will clearly describes the behavior that a user can expect. This change should solve the issue of confusion that we've heard about from internal feedback.

The description of the new slot would be:

custom-content - A slot for displaying custom content. Will prevent the rendering of any default Dialog UI, except for box-shadow and corner-radius.

deprecate(dialog): deprecate the content slot

@github-actions github-actions bot added the docs Issues relating to documentation updates only. label Dec 16, 2024
…nto dit13711/10323-rename-dialog-content-slot
Copy link
Contributor

github-actions bot commented Jan 3, 2025

This PR has been automatically marked as stale because it has not had recent activity. Please close your PR if it is no longer relevant. Thank you for your contributions.

@github-actions github-actions bot added the Stale Issues or pull requests that have not had recent activity. label Jan 3, 2025
…nto dit13711/10323-rename-dialog-content-slot
@github-actions github-actions bot removed the Stale Issues or pull requests that have not had recent activity. label Jan 18, 2025
@DitwanP DitwanP requested a review from driskull January 28, 2025 18:25
Copy link
Member

@driskull driskull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code changes look good.

Can we add a screenshot test and also test this across all browser to make sure it works?

Can we change the PR title to:

feat(dialog): add 'custom-content' slot and deprecate 'content' slot

@DitwanP DitwanP changed the title docs: deprecate add custom-content slot for dialog component feat(dialog): add 'custom-content' slot and deprecate 'content' slot Jan 28, 2025
@github-actions github-actions bot added the enhancement Issues tied to a new feature or request. label Jan 28, 2025
@DitwanP
Copy link
Contributor Author

DitwanP commented Jan 28, 2025

Looks to be working in all browsers:
image

Copy link
Member

@driskull driskull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Nice! Good to know that nesting slots like this works.

@DitwanP DitwanP added skip visual snapshots Pull requests that do not need visual regression testing. and removed skip visual snapshots Pull requests that do not need visual regression testing. labels Jan 28, 2025
@DitwanP DitwanP changed the title feat(dialog): add 'custom-content' slot and deprecate 'content' slot feat(dialog): add 'custom-content' slot Jan 28, 2025
@DitwanP DitwanP merged commit a132d00 into dev Jan 28, 2025
18 checks passed
@DitwanP DitwanP deleted the dit13711/10323-rename-dialog-content-slot branch January 28, 2025 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Issues relating to documentation updates only. enhancement Issues tied to a new feature or request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants